Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Card, Deck and README.md with test files #11

Open
wants to merge 50 commits into
base: master
Choose a base branch
from
Open

Card, Deck and README.md with test files #11

wants to merge 50 commits into from

Conversation

grissett83
Copy link

No description provided.

… and test functions to verify functionality.
…nd method to modify the amount of money the player has, plus added ValueError is the players money attempts to drop below zero. Also added extensive README.md with planned class layout and To Do list for each class.
…s methods for the new game class. Added docstrings to some classes
… pushes. Added method to determine if a player can split based on the rules set out in game options. If split_by_rank is set to true player can only split cards with the same rank.
… class. Also added takes_hit function to dealer to update dealer hand. Updated test files to reflect changes.
…odify the gameoptions object to reflect selections made in the options menu.
…xed bug where push payed out incorrect amount. Fixed formatting on player hand after split.
… Added docstrings to all methods which were missing them.
…ethod, no longer requiring dealer's upcard as an argument.
… value option, fixed error in test file where payout was not correctly predicted.
…so fixed bug that caused split hands where the second hand busted and first hand should have won to not pay out.
…ults regardless of whether the dealer had blackjack.
…k from the main blackjack directory. Added test-requirements.txt file and a run-instructions.txt file
…I could. Moved a couple of deeper nested ifs into new functions. Changed all self.game.player.hand and self.game.dealer.hand references to the new game property self.game.<player/dealer>_hand
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant